Merged
Conversation
* Merge Pull request from main to staging (#249) * update: api v1 * Merge Pull request from main to channels #248 * add: implement bristol cable channels in ChannelsController and update routes * add: enable Rack::Attack middleware for enhanced request throttling * update: No-Boost Channel --------- Co-authored-by: Min Khant Kyaw <minkhantkyawmdy35@gmail.com> Co-authored-by: Nyan Lin Htut <nyanlinhtut925@gmail.com> * remove comment out wait list for channel --------- Co-authored-by: NangKhatMang <50686926+NangKhatMang@users.noreply.github.com> Co-authored-by: Min Khant Kyaw <minkhantkyawmdy35@gmail.com> Co-authored-by: Nyan Lin Htut <nyanlinhtut925@gmail.com> Co-authored-by: NangKhatMang <nkm.nangkhatmang@gmail.com>
There was a problem hiding this comment.
Pull request overview
This pull request merges updates from the main branch, incorporating several important changes including terminology updates, infrastructure improvements, new API endpoints, and feature enablement.
Changes:
- Renamed "Automatic Search Opt-in" to "Automatic Search Opt-out" across the entire codebase with a migration task for existing data
- Updated Docker infrastructure: removed "mvp" branding, upgraded Node.js from 14.x to 20.x LTS, and modernized the build process
- Added two new API endpoints for Find Out channel collections (catch_up and speak_out)
- Enabled Bluesky bridge scheduler functionality and channel code generation UI
- Enhanced documentation in README with development setup instructions and code quality tools
Reviewed changes
Copilot reviewed 19 out of 19 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| lib/tasks/server_setting_update_search_opt_out.rake | New rake task to migrate existing ServerSetting from old to new name |
| lib/tasks/insert_server_setting_data.rake | Updated seed data to use new "Automatic Search Opt-out" naming |
| docker-compose.yml | Removed "mvp" branding from container and volume names |
| db/seeds/01_server_setttings.rb | Updated seed configuration with new setting name |
| config/routes/api_v1.rb | Added find_out_catch_up and find_out_speak_out API endpoints |
| config/data/findout/starter_pack_list.json | Removed first entry (id: "1") from starter pack list |
| app/views/wait_lists/index.html.haml | Uncommented "Generate channel code" button |
| app/views/server_settings/index.html.haml | Updated tooltip condition to use new setting name |
| app/models/user.rb | Added documentation for update_all_discoverability method |
| app/models/server_setting.rb | Updated search_opt_out_filter? to check new setting name |
| app/models/account.rb | Added documentation for update_all_discoverability method |
| app/jobs/scheduler/follow_bluesky_bot_scheduler.rb | Uncommented Bluesky bridge logic to enable the feature |
| app/controllers/wait_lists_controller.rb | Removed channel_type filter to show all wait list entries |
| app/controllers/server_settings_controller.rb | Updated base_features array with new setting name |
| app/controllers/api/v1/channels_controller.rb | Added new channel collection endpoints and constants |
| README.md | Added development setup, prerequisites, and code quality tools documentation |
| Dockerfile.backup | Updated Ruby version to 3.3.0, improved ENV declarations, updated bundler version |
| Dockerfile | Modernized Node.js installation, optimized layer caching, upgraded to Node 20.x LTS |
| .github/workflows/codeql.yml | Updated CodeQL action from v3 to v4 |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.